Split off CodeFixes from analyzer #831
Conversation
.NET8.0 says there should be no Workspace dependency in an analyzer. CodeFixes should only be loaded by an editor.
manfred-brands
force-pushed
the
issue/677_NET8.0SDK
branch
from
2c83426 to
0eddfe6
Compare
|
@mikkelbu This splits the analyzer into two projects as per guidelines. |
There was a problem hiding this comment.
Great work @manfred-brands. I checked the changes and the renamings and all look good. I also tested it out in VS and it works. So I'll be merging this now
|
@manfred-brands Out of curiosity, where does it say so? The only thing I could find was this sample: https://github.com/dotnet/samples/tree/main/csharp/roslyn-sdk/Tutorials/MakeConst. |
|
@Bartleby2718 It isn't as much that I found a description of it, but it was reported by Roslyn analyzers RS1022: Do not use types from Workspaces assembly in an analyzer and RS1038 The tutorial uses a template that creates a separate assemblies. |
.NET8.0 says there should be no Workspace dependency in an analyzer. CodeFixes should only be loaded by an editor.
Fixes #677